-
Notifications
You must be signed in to change notification settings - Fork 245
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Add warning log to Hashicorp Vault resolveSecret function #3535
feat: Add warning log to Hashicorp Vault resolveSecret function #3535
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are always happy to welcome new contributors ❤️ To make things easier for everyone, please make sure to follow our contribution guidelines, check if you have already signed the ECA, and relate this pull request to an existing issue or discussion.
...mmon/vault/vault-hashicorp/src/main/java/org/eclipse/edc/vault/hashicorp/HashicorpVault.java
Outdated
Show resolved
Hide resolved
...mmon/vault/vault-hashicorp/src/main/java/org/eclipse/edc/vault/hashicorp/HashicorpVault.java
Show resolved
Hide resolved
took myself off as a reviewer, because the changeset isn't so huge :) |
Codecov ReportAll modified lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #3535 +/- ##
==========================================
- Coverage 72.24% 63.90% -8.34%
==========================================
Files 863 863
Lines 17306 17309 +3
Branches 986 986
==========================================
- Hits 12502 11061 -1441
- Misses 4392 5873 +1481
+ Partials 412 375 -37
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
@awellnitz-materna please fix the failing check and I will merge the PR |
...mmon/vault/vault-hashicorp/src/main/java/org/eclipse/edc/vault/hashicorp/HashicorpVault.java
Outdated
Show resolved
Hide resolved
@jimmarino I have not customized any dependency. Can you tell me how to regenerate the dependency file? |
for the dependencies you can check the failing run of the license check. It suggests which the updated content of the Check |
What this PR changes/adds
Add warning log to Hashicorp Vault
resolveSecret
function.Why it does that
If the secret cannot be resolved, it is currently difficult to find out why.
The log should help the developer to find the bug faster.
Linked Issue(s)
Closes #3462